-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add ProcGoalEnv. #30
Add ProcGoalEnv. #30
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Docstrings need to be clarified but otherwise LGTM
src/seals/util.py
Outdated
@@ -118,3 +118,39 @@ def sample_distribution( | |||
def one_hot_encoding(pos: int, size: int) -> np.ndarray: | |||
"""Returns a 1-D hot encoding of a given position and size.""" | |||
return np.eye(size)[pos] | |||
|
|||
|
|||
def grid_transition_fn( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've commented on this before in #23 -- make sure you address those comments and merge the previous PR before this one.
Co-authored-by: Adam Gleave <adam@gleave.me>
Codecov Report
@@ Coverage Diff @@
## master #30 +/- ##
===========================================
+ Coverage 96.44% 100.00% +3.55%
===========================================
Files 21 22 +1
Lines 619 643 +24
===========================================
+ Hits 597 643 +46
+ Misses 22 0 -22
Continue to review full report at Codecov.
|
Co-authored-by: Adam Gleave <adam@gleave.me>
No description provided.